Create new index for tracking Asset metadata #2445
Conversation
| } | ||
|
|
||
| #[tokio::test] | ||
| #[ignore] // TODO: Need to be able to mint assets for this test to work |
There was a problem hiding this comment.
I'm not quite sure how to enable permissions to be able to mint assets for the purposes of this test
maschad
requested review from
acerone85,
netrome,
AurelienFT,
rymnc and
rafal-ch
| .checked_sub(*val) | ||
| .ok_or(anyhow::anyhow!("Asset count overflow")) |
There was a problem hiding this comment.
Is it possible that if we start indexing from now and not the past we end-up with a fail here because we would find a burn on a token with 0 (because we saw no mint before) ?
| let burn_amount = 50; | ||
|
|
||
| // Mint coins first | ||
| let mint_tx = TransactionBuilder::mint( |
There was a problem hiding this comment.
I think this is not how you mint assets. From the spec : https://github.com/FuelLabs/fuel-specs/blob/master/src/tx-format/transaction.md#transactionmint this transaction is only used by the block producer that's why it's not accepted in the pool.
It's currently used for block rewards I think and it has this much fields for future usage.
However I don't know how to mint asset with the Native assets I never used it but I think it should be that way that you should mint your coins (using script or calls). Maybe someone else in the @FuelLabs/client has more details.
Linked Issues/PRs
Closes Add indexation
AssetId -> (ContractId, SubId)#2425Related to Consider
assetsin check-account utility fuels-ts#3373Description
This introduces a new GraphQL endpoint that returns info about an asset.
The off-chain worker indexes
MintandBurnevents to store this metadata about anAssetIdChecklist
Before requesting review
After merging, notify other teams
[Add or remove entries as needed]